Skip to content

feat: return modal's close button#2948

Merged
MartinCupela merged 2 commits intomasterfrom
fix/modal-glitches
Feb 16, 2026
Merged

feat: return modal's close button#2948
MartinCupela merged 2 commits intomasterfrom
fix/modal-glitches

Conversation

@MartinCupela
Copy link
Contributor

🎯 Goal

  • refactor: change CSS variable name --control-play-control-bg-inverse to --control-play-control-bg
  • fix: return modal-close-button as opt-in and remove modal inner box shadow
  • fix: the click on overlay was not closing the modal

🛠 Implementation details

Returning close button is necessary for ModalGallery as there is not clear visual hint how to close the gallery. Clicking on transparent parts of modal inner does not close the gallery even though these parts look like overlay.

🎨 UI Changes

The close button is now of ghost circular variant

image

@github-actions
Copy link

Size Change: +63 B (+0.02%)

Total Size: 338 kB

Filename Size Change
./dist/cjs/index.js 199 kB +84 B (+0.04%)
./dist/css/index.css 33.3 kB -21 B (-0.06%)
ℹ️ View Unchanged
Filename Size
./dist/audioProcessing-BbOs2wMd.js 1.32 kB
./dist/Channel-B8wHoVv2.js 21.7 kB
./dist/ChannelPreview-CGB3q1lM.js 7.11 kB
./dist/cjs/emojis.js 2.97 kB
./dist/cjs/experimental.js 3.62 kB
./dist/cjs/mp3-encoder.js 1.27 kB
./dist/css/emojis.css 113 B
./dist/css/v2/emoji-mart.css 1.84 kB
./dist/css/v2/emoji-replacement.css 300 B
./dist/css/v2/index.css 39.4 kB
./dist/css/v2/index.layout.css 22.8 kB
./dist/limits-CtufnTQO.js 3.37 kB

compressed-size-action

@MartinCupela MartinCupela merged commit 62a27e8 into master Feb 16, 2026
4 of 5 checks passed
@MartinCupela MartinCupela deleted the fix/modal-glitches branch February 16, 2026 16:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants